I'm fine with putting this here for now, since there is other FIP related functions in this repo. But, we should probably refactor all FIP functions to a separate "AIND-FIP-UTILS repo" so that its easy to use for pavolvian, and VR-foraging data.

• Made a separate issue for the refactor. Its probably not worth doing until we are entirely sure the NWB files from the modular pipeline are structured the same: Factor out all FIP functions to a FIP specific utils repo #89

yes, all that sounds reasonable. somehow i'm failing tests (it's not linting) which is why this PR also fell to the wayside.

i'll see if i can sort it out then i assume i can merge as long as the tests pass?

TODO for rachel:

• fix the test fails.
• change the names of the functions back-- have your function be a different name so when we refactor, it will cause less disruptions.
• check with alex again before merging in PR